Skip to content

feat: add option to dump sanity check results into JSON#167

Merged
ktro2828 merged 3 commits into
mainfrom
feat/sanity/dump-json
Aug 18, 2025
Merged

feat: add option to dump sanity check results into JSON#167
ktro2828 merged 3 commits into
mainfrom
feat/sanity/dump-json

Conversation

@ktro2828

@ktro2828 ktro2828 commented Aug 17, 2025

Copy link
Copy Markdown
Collaborator

What

This PR adds a feature to dump the results of t4sanity into JSON file with -o; --output option:

$ t4sanity <DATA_ROOT> -o results.json -iw

>>>Sanity checking...: 2it [00:00, 21.54it/s]
...

Then a JSON file named results.json will be generated:

[
  {
    "dataset_id": "dataset1",
    "version": 2,
    "status": "ERROR",
    "message": "bbox must be (xmin, ymin, xmax, ymax) and xmin <= xmax && ymin <= ymax: (1532, 198, 1440, 265)"
  },
  {
    "dataset_id": "dataset2",
    "version": 1,
    "status": "WARNING",
    "message": "Category token is empty for surface ann: 0c15d9c143fb2723c16ac7e0c735b0a8"
  }
]

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
@github-actions github-actions Bot added documentation Improvements or additions to documentation new-feature New feature or request labels Aug 17, 2025
@github-actions

github-actions Bot commented Aug 17, 2025

Copy link
Copy Markdown
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
2302 1562 68% 50% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: ff1640c by action🐍

@ktro2828 ktro2828 force-pushed the feat/sanity/dump-json branch from aa7fb6b to eecc06a Compare August 18, 2025 00:32
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
@ktro2828 ktro2828 force-pushed the feat/sanity/dump-json branch from eecc06a to ff1640c Compare August 18, 2025 01:51
@ktro2828 ktro2828 merged commit 758e5c8 into main Aug 18, 2025
4 of 5 checks passed
@ktro2828 ktro2828 deleted the feat/sanity/dump-json branch August 18, 2025 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation new-feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant